Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GH Actions: sync PR improvements #5649

Merged
merged 2 commits into from
Jul 28, 2023
Merged

GH Actions: sync PR improvements #5649

merged 2 commits into from
Jul 28, 2023

Conversation

MetRonnie
Copy link
Member

@MetRonnie MetRonnie commented Jul 25, 2023 •
edited
Loading

  • Try fast-forward merge of 8.*.x branch into master and push if that's successful.
  • Otherwise always create PR from sync branch instead of creating a PR from 8.*.x into master direct

@MetRonnie MetRonnie added the infrastructure GH Actions, Codecov etc. label Jul 25, 2023
@MetRonnie MetRonnie mentioned this pull request Jul 25, 2023
Merged
@MetRonnie
Copy link
Member Author

Test failures unrelated - pending #5648

@oliver-sanders oliver-sanders added this to the cylc-8.2.1 milestone Jul 28, 2023
@oliver-sanders oliver-sanders merged commit f22b81e into cylc:8.2.x Jul 28, 2023
2 of 7 checks passed
@oliver-sanders
Copy link
Member

oliver-sanders commented Jul 28, 2023 •
edited
Loading

Merged this and triggered the action.

Ended up with two PRs for the same changes, which I guess makes sense (one for merge, one for dispatch, both FF)

@MetRonnie MetRonnie deleted the actions branch August 1, 2023 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliver-sanders oliver-sanders oliver-sanders approved these changes

Assignees
No one assigned
Labels
infrastructure GH Actions, Codecov etc.
Projects
None yet
Milestone
cylc-8.2.1
Development

Successfully merging this pull request may close these issues.
2 participants
@MetRonnie @oliver-sanders